Skip to content

Reduce bundle size of this package, fix CI#494

Closed
tbantle22 wants to merge 5 commits intoliuliu-dev:mainfrom
dolthub:taylor/bundle
Closed

Reduce bundle size of this package, fix CI#494
tbantle22 wants to merge 5 commits intoliuliu-dev:mainfrom
dolthub:taylor/bundle

Conversation

@tbantle22
Copy link

@tbantle22 tbantle22 commented Feb 24, 2026 

  ┌─────────────┬─────────┬────────┬───────────┐
  │ Metric      │ Before  │ After  │ Reduction │
  ├─────────────┼─────────┼────────┼───────────┤
  │ ESM raw     │ 12.0 MB │ 92 KB  │ 99.2%     │
  │ CJS raw     │ 12.0 MB │ 93 KB  │ 99.2%     │
  │ ESM gzipped │ 2.2 MB  │ 2.6 KB │ 99.8%     │
  └─────────────┴─────────┴────────┴───────────┘

For reducing bundle size:

  • Removes large dependencies that are easy to implement inline
  • Remove unused dependencies and code

For CI:

  • Github actions workflow file used the wrong path so it wasn't running on PRs
  • Removed next (not really necessary since it doesn't use pages)
  • Get eslint and tests running and passing

@liuliu-dev
Copy link
Owner

liuliu-dev commented Mar 26, 2026

Hi @tbantle22 👋 , happy to see you here!!!
Sorry I missed this PR for so long...
I did something similar and merged a huge PR yesterday to bump a lot of dependencies and fix CI (and found out the CI was a mess and never worked 🫠 ). Then I started closing all the Dependabot PRs and found yours here.
I will close it but thanks for taking the time to put up this PR!

@liuliu-dev liuliu-dev closed this Mar 26, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@tbantle22 @liuliu-dev